[Documentation/99] 윤지윤 문제3번 이슈추가#101
Conversation
yoonjiyoonjiyoon
added
the
documentation
Summary of ChangesHello @yoonjiyoonjiyoon, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! 이 PR은 저장소의 전반적인 파일 구조를 github_tutorial 디렉토리 아래로 재편성하는 광범위한 변경 사항을 포함합니다. 이는 기존 학습 자료들의 조직화를 개선하고, 새로운 실습 답안 문서와 C++ 소스 코드 파일을 추가하며, 일부 오래된 실습 문서를 제거하는 것을 목표로 합니다. 전반적으로 저장소의 콘텐츠를 업데이트하고 구조를 명확히 하는 데 중점을 둡니다. Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
|
good..? |
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
안녕하세요. GitHub 튜토리얼 실습 PR에 대한 리뷰입니다.
전반적으로 실습을 잘 진행하셨지만, 몇 가지 수정이 필요한 부분이 있습니다.
가장 중요한 점은 파일 위치입니다. 윤지윤-전기공학과.md 파일이 github_tutorial/practice/ 디렉토리가 아닌 github_tutorial/에 위치해 있습니다. PR 설명과 실습 안내에 따라 올바른 경로로 파일을 이동시켜 주세요.
또한, 실습 내용과 관련이 없어 보이는 윤지윤.cpp 파일이 함께 추가되었습니다. 만약 실수로 포함된 파일이라면 이번 PR에서 제외하는 것을 권장합니다. 만약 의도된 파일이라면, 컴파일이 불가능한 치명적인 오류와 코드 스타일 문제를 수정해야 합니다.
자세한 내용은 각 파일에 남긴 개별 코멘트를 확인해주세요.
| cout << "안녕하세요" << end1; | ||
| cout << "안녕하세요1" << end1; | ||
| cout << "안녕하세요3" << end1; |
There was a problem hiding this comment.
컴파일 오류가 발생합니다. end1은 정의되지 않은 식별자입니다. 줄바꿈을 위해서는 endl을 사용해야 합니다. std::endl의 오타로 보입니다.
| cout << "안녕하세요" << end1; | |
| cout << "안녕하세요1" << end1; | |
| cout << "안녕하세요3" << end1; | |
| cout << "안녕하세요" << endl; | |
| cout << "안녕하세요1" << endl; | |
| cout << "안녕하세요3" << endl; |
| @@ -0,0 +1,12 @@ | |||
| #include <iostream> | |||
|
|
|||
| using namespace std; | |||
There was a problem hiding this comment.
using namespace std;를 전역 범위에서 사용하는 것은 이름 충돌(name collision)의 위험이 있어 권장되지 않습니다. 특히 헤더 파일에서는 절대 사용해서는 안 되며, 소스 파일에서도 가급적 std::cout, std::endl과 같이 std:: 접두사를 붙여 명시적으로 사용하는 것이 좋습니다.
|
|
||
| ## 문제 1. 브랜치 분기/병합 및 로그 그래프 캡처 | ||
|
|
||
| - |
There was a problem hiding this comment.
Markdown 문법에 맞지 않는 부분입니다. 리스트 항목으로 이미지를 표시하려면 -와 ! 사이에 공백이 한 칸 필요합니다. 이대로는 이미지가 정상적으로 렌더링되지 않을 수 있습니다.
| - | |
| -  |
| ## 문제 3. Issue 생성 + PR 승인 후 병합 (컨벤션 준수) | ||
|
|
||
| PR 정보(예시, 수업과 내용이 상이함. 수업 내용을 기준으로 작성 할 것) | ||
| - 링크: https://github.com/EliseEgkart/github_tutorial_UNITA_2026/pull/7 No newline at end of file |
There was a problem hiding this comment.
불필요한 공백이 있습니다. 콜론(:) 뒤에 공백이 두 칸 들어가 있습니다. 일관성을 위해 한 칸만 사용하는 것이 좋습니다.
| - 링크: https://github.com/EliseEgkart/github_tutorial_UNITA_2026/pull/7 | |
| - 링크: https://github.com/EliseEgkart/github_tutorial_UNITA_2026/pull/7 |
1 participant
✅ 공통 (모든 PR에 작성)
📜 [documentation] 문서 PR